Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support multiple bridges in one monitor #262

Merged
merged 31 commits into from
Jan 10, 2020
Merged

Support multiple bridges in one monitor #262

merged 31 commits into from
Jan 10, 2020

Conversation

patitonar
Copy link
Contributor

Closes #138

Follows the approach described here.

@patitonar
Add support to disable validator balance check in monitor
40269d2
@patitonar
Convert validator address to checksum address
a4b049d
@patitonar
Avoid extra calls if tx limit is zero in validators checks
441c944
@patitonar
Remove bridge specific logic from index.js in monitor
4e3181e
@patitonar
Store monitor results in responses/bridge_name
5ce2b7b
@patitonar
Add example configs for monitor
639c319
@patitonar
Rename checkDocker.sh to getBridgeStats.sh
ad643f5
@patitonar
Run monitor check-all for every env file
dc4aefc
@patitonar
Update monitor deployment env file
392af8b
@patitonar
Copy monitor bridge config files in deployment script
ffeee56
@patitonar
fix monitor deployment e2e test
34f6fa3
@patitonar
Update env vars for monitor e2e tests
c9d6c45
@patitonar
Fix monitor e2e base urls
ce72dd6
@patitonar
Run monitor check job after deployment
5ed21f4
@patitonar
Add example env for deployment-monitor e2e tests
a205fbe
@patitonar
fix copy example env for monitor deployment e2e
fe0baee
@patitonar
Only copy env files if monitor already deployed
34128a5
@patitonar
Merge branch 'develop' into 138-multiple-bridges-one-monitor
6708d78 
# Conflicts:
#	monitor/index.js
#	monitor/validators.js
@patitonar
Add monitor deployment configuration docs
c473d46
@patitonar
remove unused variables
551b81a
@patitonar patitonar added deployment related to deployment playbooks/scripts/configs monitor related to TokenBridge monitor labels Jan 6, 2020
@patitonar patitonar requested a review from akolotov January 6, 2020 20:42
@patitonar patitonar self-assigned this Jan 6, 2020
@patitonar patitonar changed the title Support multiple bridges one monitor Support multiple bridges in one monitor Jan 7, 2020
@patitonar patitonar mentioned this pull request Jan 7, 2020
Closed
akolotov
akolotov reviewed Jan 7, 2020
View reviewed changes
@akolotov akolotov merged commit 65dd131 into develop Jan 10, 2020
@akolotov akolotov deleted the 138-multiple-bridges-one-monitor branch January 10, 2020 12:55
@akolotov akolotov mentioned this pull request Jan 13, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akolotov akolotov akolotov approved these changes

Assignees

@patitonar patitonar

Labels
deployment related to deployment playbooks/scripts/configs monitor related to TokenBridge monitor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@patitonar @akolotov